-
-
Notifications
You must be signed in to change notification settings - Fork 160
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix(esm): resolve absolute paths #544
fix(esm): resolve absolute paths #544
Conversation
Could this also be addressing #541? |
@fardjad Would you mind adding a test? |
2a65b01
to
5cbc7e3
Compare
5cbc7e3
to
5d7ca18
Compare
@privatenumber Of course! I updated my PR. I wasn't sure where the best place to add the test would be. I decided to put it under the |
18e18ce
to
5a5ca97
Compare
That works, but I consolidated it into the smoke test—previously, our tests would take super long from spawning so many processes. Thanks for your contribution! |
🎉 This issue has been resolved in v4.9.2 If you appreciate this project, please consider supporting this project by sponsoring ❤️ 🙏 |
This PR attempts to fix the issue described here: #543